Skip to content

Conversation

@dmirgaev
Copy link
Contributor

No description provided.

@dmirgaev dmirgaev requested a review from a team as a code owner January 15, 2026 11:28
Copilot AI review requested due to automatic review settings January 15, 2026 11:28
@dmirgaev dmirgaev added the 26_1 label Jan 15, 2026
@dmirgaev dmirgaev self-assigned this Jan 15, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes overlapping issues between group rows and total summary in DataGrid by adjusting background-color properties for sticky columns across all themes.

Changes:

  • Changed background-color: transparent to background-color: inherit for sticky columns in group rows across all three themes (material, generic, fluent)
  • Inverted the CSS selector logic in the base gridBase file for total footer to apply background-color: transparent to non-sticky columns instead of sticky columns

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
packages/devextreme-scss/scss/widgets/material/dataGrid/_index.scss Changed sticky group cell background from transparent to inherit
packages/devextreme-scss/scss/widgets/generic/dataGrid/_index.scss Changed sticky group cell background from transparent to inherit
packages/devextreme-scss/scss/widgets/fluent/dataGrid/_index.scss Changed sticky group cell background from transparent to inherit
packages/devextreme-scss/scss/widgets/base/_gridBase.scss Inverted selector to apply transparent background to non-sticky columns in total footer

@dmirgaev dmirgaev force-pushed the 26_1__T1284612_summary branch from 9bd12d2 to 101dd1e Compare January 16, 2026 17:42
Copilot AI review requested due to automatic review settings January 16, 2026 17:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

@dmirgaev dmirgaev force-pushed the 26_1__T1284612_summary branch from 101dd1e to ba19bc5 Compare January 19, 2026 15:22
Copilot AI review requested due to automatic review settings January 22, 2026 09:03
@dmirgaev dmirgaev force-pushed the 26_1__T1284612_summary branch from ba19bc5 to fb44f69 Compare January 22, 2026 09:03
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings January 22, 2026 10:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI review requested due to automatic review settings January 22, 2026 15:16
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 18 changed files in this pull request and generated no new comments.

@dmirgaev dmirgaev force-pushed the 26_1__T1284612_summary branch from d7a7b63 to 5421d64 Compare January 22, 2026 16:59
Copilot AI review requested due to automatic review settings January 22, 2026 16:59
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 20 changed files in this pull request and generated 1 comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant